Skip to content

Ai/issue 2 devloop#7

Merged
SmileJune merged 4 commits into
mainfrom
ai/issue-2-devloop
May 24, 2026
Merged

Ai/issue 2 devloop#7
SmileJune merged 4 commits into
mainfrom
ai/issue-2-devloop

Conversation

@SmileJune

@SmileJune SmileJune commented May 24, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • New Features

    • Two-stage approval workflow: /ai approve creates PR scaffold, then /ai implement executes changes locally
    • Local DevLoop runner for automated implementation of approved PRs

  • Documentation

    • Added comprehensive runner workflow documentation with operational examples
    • Added new DevLoop issue scaffold

  • Chores

    • Enhanced duplicate proposal prevention system
    • Updated approval workflow policies and constraints

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 24, 2026 •
edited
Loading

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6bcf5a79-128a-4418-a139-123625294630

📥 Commits

Reviewing files that changed from the base of the PR and between 3abd0ab and 319be04.

📒 Files selected for processing (8)
  • AGENTS.md
  • AI_POLICY.md
  • docs/ai-ideas/issue-2.md
  • docs/devloop-runner.md
  • scripts/ai/implement_idea.py
  • scripts/ai/prompts/unified_employee.md
  • scripts/ai/propose_idea.py
  • scripts/devloop/runner.py
📝 Walkthrough

Walkthrough

This PR implements a two-stage local DevLoop execution workflow. After human approval via /ai approve creates a PR scaffold, the local runner polls GitHub for /ai implement comments, validates approval, executes Codex in a sandboxed environment, and optionally commits/pushes changes back to the PR branch with enforced safety constraints.

Changes

DevLoop Local Implementation Workflow

Layer / File(s) Summary
Policy and approval workflow definition
AGENTS.md, AI_POLICY.md		 DevLoop MVP now explicitly requires /ai approve to create PR scaffolds and /ai implement to trigger local Codex execution. Defines allowed runner behaviors, prohibits commits without human command, and prohibits pushes to main.
Recent proposal deduplication system
scripts/ai/prompts/unified_employee.md, scripts/ai/propose_idea.py		 Retrieves recent DevLoop-labeled issues, builds a context summary, and injects it into OpenAI prompt with explicit non-repetition constraints; updates github_request return type and api_idea/load_idea signatures to accept recent context.
PR scaffold and implementation template
scripts/ai/implement_idea.py, docs/ai-ideas/issue-2.md		 PR scaffold template now uses Refs instead of Closes to avoid automatic issue closure; adds concrete Issue #2 example scaffold documenting scope constraints (UI/docs only) and implementation restrictions.
DevLoop runner documentation
docs/devloop-runner.md		 Specifies end-to-end runner workflow from /ai implement through codex exec --sandbox workspace-write, environment variables, three operational modes (dry-run, execute-only, execute-with-commit), and safety constraints (no merges; no pushes to main; forbidden paths).
DevLoop runner script implementation
scripts/devloop/runner.py		 Polling runner that detects /ai implement comments, validates approvers and workspace cleanliness, fetches PR branches, executes Codex locally, filters changes against forbidden paths, optionally commits/pushes, and posts GitHub status comments with results.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~60 minutes

Poem

🐰 A rabbit hops through two-stage flows,
/ai approve plants the scaffold row,
/ai implement triggers Codex's dance,
While safety guards prevent the wrong branch chance,
No proposals repeat, the runner polls keen,
Building DevLoops locally—the finest foreseen! 🌟

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ai/issue-2-devloop

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@SmileJune SmileJune merged commit d970d80 into main May 24, 2026
1 of 2 checks passed
@coderabbitai coderabbitai Bot mentioned this pull request May 24, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@SmileJune